home *** CD-ROM | disk | FTP | other *** search
Text File | 1989-09-27 | 19.2 KB | 735 lines | [TEXT/MPS ] |
- Known bugs/issues/enhancement requests in MacApp 2.0B9
- 9/27/89
- Compiled by Richard Rodseth (MACAPP.TEST)
- MacApp Quality Assurance
- Apple Computer, Inc.
-
- - I have included only the bug number and summary.
- Some of the summaries are rather cryptic.
- Contact me if you are concerned about a particular bug.
- - There are some omissions. In particular, bugs in the sample programs,
- ViewEdit and Mouser are not included.
- If you are aware of serious bugs which are not included,
- let me know right away.
- - Not all of these bugs will be fixed in 2.0 final.
- Some are enhancement requests; some are debatable;
- some raise broader architectural issues.
- - Some of these bugs are still under investigation.
- Don't infer too much from the summary.
- We may decide after investigation that something isn't a bug after all.
-
-
- BRC No: 29431
- -------------
- Adorned shadow on a control is not printed.
-
- BRC No: 31550
- -------------
- After wraparound in a TEView with non-zero bottom inset, only part
- of the second line is displayed.
-
- BRC No: 34719
- -------------
- TView.CenterInSuperView and other layout methods would be nice.
- TWindow.Center could be an override.
-
- BRC No: 36018
- -------------
- TView should override Clone in order to clone subviews.
-
- BRC No: 39108
- -------------
- MacApp should not use the console device.
-
- BRC No: 40834
- -------------
- The application signature should be a parameter to IApplication, and
- the default DoMakeDocument should use this value. Alternatively,
- kSignature should be removed from Nothing.p
-
- BRC No: 41271
- -------------
- Posemodally should be a window method (?)
-
- BRC No: 41273
- -------------
- It is unnecessarily difficult to filter the list of files shown in the
- Standard file dialog.
-
- BRC No: 45983
- -------------
- Design flaw in TDocument.DoMenuCommand (use of fSaveExists to
- determine whether RequestFileName should be called)
-
- BRC No: 45985
- -------------
- TTEView.ScrollSelectionIntoView fails if the TEView is not visible,
- because it relies on focusing.
-
- BRC No: 46096
- -------------
- Printing Demotext documents to an imagewriter will occassionally
- result in the I-Beam cursor getting printed also.
-
- BRC No: 46100
- -------------
- During document view creation, TView.AdjustSize does not get
- called if the view is not attached to a printhandler.
-
- BRC No: 46103
- -------------
- An extra blank page is printed if fSizedeterminer = sizeFillPages and
- fPrintHandler.fFixedSizePages = FALSE. In this case, ComputeSize
- should loop calling DoBreakFollowing.
-
- BRC No: 46385
- -------------
- Under A/UX, the cursor changes to the watch while tracking, even in
- menus and controls.
-
- BRC No: 46712
- -------------
- TWindow.AdaptToScreen doesn't account for window location - the
- bottom right can end up off-screen.
-
- BRC No: 48615
- -------------
- On a Kanji system, the font doesn't get set to a Kanji font in an
- unstyled version of DemoText.
-
- BRC No: 48945
- -------------
- TTEView.SynchView only updates the text if the line heights have
- changed. If a program modifies the text directly, ForceRedraw has to
- be called.
-
-
-
- BRC No: 49695
- -------------
- TEvtHandler.DoCreateViews doesn't work if views are specified in
- breadth-first order.
-
- BRC No: 49702
- -------------
- If an application is launched from a floppy which is then ejected,
- when the alert requests that the disk be inserted, repeatedly
- pressing command-period causes a crash.
-
- BRC No: 49704
- -------------
- Unique signatures should be reserved for the sample applications and
- supported by the build process.
-
- BRC No: 49705
- -------------
- Without the DA Handler, the close menu item closes a DA window,
- but Command-W does not.
-
- BRC No: 49706
- -------------
- In TEViews, right justified italics is chopped off at the edge of the
- view.
-
- BRC No: 49794
- -------------
- The -linkmap and -linkXRef options to MABuild are not included in
- the Commando dialog.
-
- BRC No: 49795
- -------------
- The text item in the default about box is a bit too small for the
- Kanji default font.
-
- BRC No: 50431
- -------------
- When switching out in MultiFinder, MacApp commits command
- objects which affect the clipboard, rather than checking if the scrap
- has changed when switching back in.
-
-
- BRC No: 50432
- -------------
- It is potentially problematic having Page SetUp as an undoable
- command, since the view and printer driver context can change.
-
- BRC No: 50433
- -------------
- When quitting an application with multiple unsaved documents, the
- save alerts are displayed before the windows are updated. As such,
- the user cannot see the contents of the document which is in
- question of being saved.
-
- BRC No: 51223
- -------------
- TView.WriteToDeskScrap doesn't write subviews.
-
- BRC No: 51224
- -------------
- TDocument.DoMakeViews disregards the print info saved with a
- document, and restores the default print info.
-
- BRC No: 51350
- -------------
- TWindow.Center is inaccurate.
-
- BRC No: 51386
- -------------
- The comment for fIdleFreq in UMacApp.p is incorrect. (0 does not
- mean no idle)
-
- BRC No: 51388
- -------------
- A -needsMC68020 application crashes before user code on a 68000
- machine (If built with MPW 3.0 Pascal)
-
- BRC No: 51503
- -------------
- MABuild -statusonly fails if the Makeit file does not exist.
-
- BRC No: 51504
- -------------
- The MacApp debugger should accept expressions in the "D" option, and
- check for invalid input.
-
- BRC No: 51505
- -------------
- MacApp samples crash into Macsbug with CHK break, if built with the
- options:
- "-DebugTheDebugger -Bottleneck -RangeCheck".
-
- BRC No: 51617
- -------------
- TApplication.TrackMouse method is wrong if theCommand.fView is
- NIL during all tracking phases.
-
- BRC No: 51641
- -------------
- If you have a TDialogView with multiple TEditText views, and the
- views have different fMaxChars lengths, then you will get a bogus
- program break when tabbing from a smaller TEditText view to a
- larger one.
-
- BRC No: 51727
- -------------
- MABuild -autobuild -nodebug -notemplateviews fails, because the
- target folder name is '.'
-
-
- BRC No: 51728
- -------------
- Application hangs if Print… is chosen while stopped in the debugger.
-
- BRC No: 51770
- -------------
- If aWindow.fIsClosable for any open window is FALSE, MacApp gets
- caught in an infinite loop during TApplication.Close.
-
- BRC No: 51880
- -------------
- In TApplication.PerformCommand (HdlDoIt), fCmdNumber is
- referenced after the command object has been freed.
-
- BRC No: 52716
- -------------
- UMacApp.TApplication.p should release mbar resource rather than
- calling DisposIfHandle.
-
- BRC No: 52717
- -------------
- The comment in TSortedList.Compare should read "compares TWO
- items" not "compares to items".
-
- BRC No: 52718
- -------------
- The ProgramBreak in TGridView.SetColWidth refers to SetColHeight
- instead of SetColWidth.
-
- BRC No: 52719
- -------------
- ITEView calls BeInScroller(GetScroller(FALSE)), which is
- inappropriate when a TEView is nested in other views. TTEView.IRes
- doesn't call BeInScroller.
-
- BRC No: 52720
- -------------
- TEvtHandler.DoMultiClick uses an OR instead of an AND.
-
- BRC No: 52836
- -------------
- The default window size limits limit the size of a window to the
- size of the main screen, rather than the gray region.
-
- BRC No: 52837
- -------------
- TPopup.DoMouseCommand puts the top left corner of the selected
- item next to the labelRect rather than over the menuRect. For
- standard TPopups, this is the same.
-
- BRC No: 52865
- -------------
- Program Break when user enters close to maximum number of
- characters in EditText (and NumberText) fields.
-
- BRC No: 53537
- -------------
- In TDialogTEView.InstallEditText and TEditText.SetText, SynchView
- should be called after RecalcText.
-
- BRC No: 53538
- -------------
- TDialogTEView.RecalcText prevents autoscrolling of right-justified
- EditTexts.
-
- BRC No: 53539
- -------------
- TEditText.RestartEdit should call fTEView.ForceRedraw to invalidate
- the text, rather than using InvalRect on the control area, which
- assumes fTEView's focus is the same as its own.
-
- BRC No: 53540
- -------------
- TStaticText.IRes should initialize fRsrcID to kNoResource and fIndex
- to 0. These fields could be eliminated as they are not used after the
- text is set.
-
- BRC No: 53541
- -------------
- There are some unused local variables in TTEView.CalcMinSize.
-
- BRC No: 53542
- -------------
- DrawContents should not propagate failure, because an endless loop
- can result due to the update when the alert is dismissed. Silently
- failing isn't the solution, however.
-
- BRC No: 53543
- -------------
- For pageUp/pageDown keys, it would be more useful to have the
- target of the Inspector window be the object view.
-
- BRC No: 53544
- -------------
- VerboseIsObject bombs with a non-object handle like a region
- handle, because GetClassID fails.
-
- BRC No: 53545
- -------------
- TCtlMgr.WhileFocussed calls Focus unnecessarily when redraw is
- false.
-
- BRC No: 53546
- -------------
- TSScrollbar.Activate should be conditioned on IsFocussed, rather
- than relying on "gFocusedView = SELF" kludge in
- TCtlMgr.WhileFocused.
-
- BRC No: 53547
- -------------
- The FOR loop in TTEView.CalcMinSize is redundant because
- ComputeSize performs this function.
-
- BRC No: 53548
- -------------
- The "about" command name is set in TApplication.DoSetUpMenus. It
- could be done once, when the menus are read in.
-
- BRC No: 53549
- -------------
- Globals used by DoShowAboutAppFilter need to be explained/named
- g…/moved to VAR section of unit or put in interface.
-
- BRC No: 53550
- -------------
- TApplication.GetRsrcWindow calls SetWRefCon before checking
- aWMgrWindow for NIL. This assumes SetWRefCon can handle a nil
- parameter nicely.
-
- BRC No: 53551
- -------------
- TApplication.HandleUpdateEvent has an extra BEGIN..END;
-
- BRC No: 53552
- -------------
- Redundant code surrounded by {$IFC FALSE} in TPopUp.ReleasePopUp
- and TPopup.SetPopUp.
-
- BRC No: 53553
- -------------
- TDialogView.DeselectCurrentEditText is in the wrong place in the
- source. (It Should appear before DismissDialog.)
-
- BRC No: 53554
- -------------
- TPopup.IPopup should call FailResError instead of FailNILResource,
- like IRes.
-
- BRC No: 53555
- -------------
- TPopup.DoMouseCommand needs a failure handler in case DoChoice
- fails.
-
- BRC No: 53556
- -------------
- TEditText.StartEdit should exit if theTEView is nil, even in the non-
- debug case.
-
- BRC No: 53557
- -------------
- TDialogView.MakeTEView: The system call SetClikLoop _cannot_ be
- used to set the clikProc of the TERecord, because the default
- clikproc does not follow the Pascal Parameter passing conventions
- that this procedure expects.
-
- BRC No: 53558
- -------------
- TTEView.CalcSelLoc returns a bogus vertical coordinate when the
- StyledTextEdit record you pass to it has no characters in it.
-
- BRC No: 53559
- -------------
- TTEView.CalcSelLoc might be more useful if selectionRect were a
- VRect
-
- BRC No: 53560
- -------------
- Modifications to TTEView.ScrollSelectionIntoView:
-
- BRC No: 53561
- -------------
- TEditText should give more feedback than a beep when maxchars is
- exceeded. The check should be done in the Validate method.
-
- BRC No: 53562
- -------------
- There should be a VRectsNest function.
-
- BRC No: 53563
- -------------
- The description of TApplication.HandleMouseUp in UMacApp.p refers
- to the non-existent gEventInfo.
-
- BRC No: 53623
- -------------
- BusyDelay has a redundant IF statement. The comment in the
- interface file is truncated.
-
- BRC No: 53624
- -------------
- TView.DoneViewRsrc could use SetPermHandleSize instead of
- SetHandle size and FailMemError, and the setting of the
- PermAllocation flag is not strictly necessary here.
-
- BRC No: 53625
- -------------
- TView.AddSubView sets fNextHandler twice.
-
- BRC No: 53626
- -------------
- Assertion should ALWAYS call Failure, even in non-debug mode.
-
- BRC No: 53627
- -------------
- Stepping over in the debugger doesn't break at the end of a method.
- pStackSpace should be updated in %_EP
-
- BRC No: 53628
- -------------
- "Space to step over stack" doesn't work if breakpoints are set.
-
- BRC No: 53629
- -------------
- The debugger should have a way to show the current status without
- listing all the possible commands, scrolling the status out of view.
-
- BRC No: 53630
- -------------
- If the performance Monitor is on and the Performance tools are
- "E"nded, the Perfmonitor still shows as being on.
-
-
- BRC No: 53631
- -------------
- DebugPerfMonitor: The private global pPerfControl is not necessary
- as it duplicates the function of PerfControl.
-
- BRC No: 53632
- -------------
- In MABuild.script, when expanding the active environment vars into
- the make file, the line "SET SrcApp {XMakeitRedirection}" is
- duplicated unnecessarily.
-
- BRC No: 53634
- -------------
- Calc's splash screen leaves heap fragmented.
-
- BRC No: 53635
- -------------
- TGridView.Draw shouldn't subtract 1 from bottom and right of draw
- area.
-
- BRC No: 53636
- -------------
- TGridView.VPointToLastCell compares cell coordinates to
- QuickDraw coordinates (apples and oranges)
-
- BRC No: 53637
- -------------
- References to gCodeReserve etc in UMemory.p should be to
- pCodeReserve etc. Also should read pSzCodeReserve - (the total size
- of all temporary objects in memory [at that time]).
-
- BRC No: 53747
- -------------
- Attempting to select a gridview cell for which CanSelect returns
- false causes the current selection to be deselected.
-
- BRC No: 53748
- -------------
- TScrollbar should be "deadstripsuppressed"
-
- BRC No: 53750
- -------------
- NewTemplateWindow should check that the root view is a window,
- and fail gracefully if it isn't.
-
- BRC No: 53751
- -------------
- FailNIL on missing clipboard window or view gives confusing "not
- enough memory" alert.
-
- BRC No: 53752
- -------------
- It should be possible to prevent TDialogView.PoseModally from
- committing the last command if fDismisser is fCancelItem, or if the
- command requiring the dialog is itself not undoable.
-
- BRC No: 53753
- -------------
- MABuild doesn't support spaces in the "OtherViewTypesSrc = " path
- name.
-
- BRC No: 53754
- -------------
- TView.Update should have a failure handler.
-
- BRC No: 53755
- -------------
- TCtlMgr.Free calls SizeControl which calls HideControl, but
- coordinates are wrong. It would be better to set the clip region to
- zero.
-
- BRC No: 53756
- -------------
- It would be preferable for TGridView.IRes to insert the columns
- first, then the rows.
-
- BRC No: 53757
- -------------
- TTEView.Idle should not flash the cursor if the view is disabled.
-
- BRC No: 53758
- -------------
- GetSaveInfo/MakeNewCopy let the user save a document over a
- document with the same name, even if the type and creator bytes are
- different.
-
- BRC No: 53760
- -------------
- Applications have been known to crash printing multiple documents
- from the Finder.
-
- BRC No: 53815
- -------------
- MABuild doesn't support more than one user library, due to a quoting
- problem.
-
- BRC No: 53816
- -------------
- TButton.SetText(s,TRUE) stuffs the text, but also redraws the button
- outline and causes the whole thing to flash.
-
- BRC No: 53817
- -------------
- GetClassNameFromID bombs for names of length > 63
-
- BRC No: 53818
- -------------
- The call to GetEvent in TApplication.UpdateAllWindows ignores
- app4Events.
-
- BRC No: 53821
- -------------
- TWindow.ForceOnScreen doesn't.
-
- BRC No: 53922
- -------------
- TTEView.SynchView relies on ScrollSelectionIntoView to focus.
- ScrollSelectionIntoView does not attempt to focus unless there is a
- scroller.
-
- BRC No: 53923
- -------------
- TrapExists in UMacAppUtilities.inc1.p is incorrect for traps in range
- $AA00-ABFF
-
- BRC No: 53924
- -------------
- GetMenuColors in UDialog.inc1.p dereferences gdh without checking
- for nil.
-
- BRC No: 54112
- -------------
- If the DrawContents method fails after the BeginUpdate, the call to
- EndUpdate will never happen. The application can get into an infinite
- loop, displaying 'failed..not enough memory', while trying to update a
- window that can not be updated.
-
- BRC No: 54130
- -------------
- "Fix" in TTEView.StuffText gets rid of style info by setting
- runs[1].startchar, but doesn't adjust nRuns or nStyles.
-
- BRC No: 54131
- -------------
- MacAppTypes.r does not conditionally auto-include Types.r
-
- BRC No: 54132
- -------------
- UMacAppUtilities places code in segment MAIN. People using C and
- C++ libraries run out of space in MAIN.
-
- BRC No: 54284
- -------------
- PoseModally doesn't bring its window to the top when it is called.
-
- BRC No: 54358
- -------------
- Calling SwapMMUMode in a debug MacApp program with MacroMaker
- and 32-bit QD installed causes a crash.
-
- BRC No: 54444
- -------------
- TView should have GetIdentifer and SetIdentifier methods
-
- BRC No: 54445
- -------------
- TApplication.PerformCommand won't perform the same pre-
- allocated undoable command twice in succession because it checks
- IF command <> gLastCommand.
-
- BRC No: 54446
- -------------
- TWindow.Activate calls update before INHERITED Activate, so
- subviews get update messages before they're activated.
-
- BRC No: 54447
- -------------
- With a DA open in the application heap, you can enter the MacApp
- debugger, but it won't accept any keystrokes.
-
- BRC No: 54448
- -------------
- Autoscrolling in GridViews stops when the mouse's coordinates go
- outside of the view's extent.
-
- BRC No: 54449
- -------------
- TStdPrintHandler.BanishPrintDialog and FreeWmgrWindow should
- invalidate the focus when setting the port to gWorkPort
-
- BRC No: 54450
- -------------
- PinVRect is off-by-one at bottom-right
-
- BRC No: 54451
- -------------
- The documentation doesn't state that OtherViewTypesSrc should be
- set in the make file.
-
- BRC No: 54452
- -------------
- TApplication.SetTarget should check if the new target is different
- from the current target before doing its thing
-
- BRC No: 54453
- -------------
- TTranscriptView.InstallTextStyle doesn't correctly restore the port
-
- BRC No: 54454
- -------------
- TWindow.SetTitle should only call SetWTitle if the newTitle is
- different from the current title
-
- BRC No: 54455
- -------------
- TPopup.DrawPopupBox doesn't correctly include the outermost pixel
- of all the menuRect's edges in its SectRect test.
-
- BRC No: 54456
- -------------
- TPopup.SetCurrentItem (..kRedraw) fails AssumeFocused. It should
- either focus before calling GetQDExtent, or just call ForceRedraw
- instead of trying to directly redraw the popup box itself.
-
- BRC No: 54457
- -------------
- On small screens, it would be nice if the debug window could be
- brought to the front by "Show Debug Window", and prevented from
- going to the back, especially during single-stepping.
-
- BRC No: 54458
- -------------
- FreeIfObject is usually called while handling a failure - perhaps it
- should do IF IsObject(obj) THEN obj.Free ELSE ProgramBreak rather
- than signalling failure with FailNonObject.
-
- BRC No: 54459
- -------------
- The last two pages of an image were printed blank on a LaserWriter
- IISC.
-
- BRC No: 54460
- -------------
- A file cannot be made -d option dependent in .MAMake, nor can
- separate folders be created following -d options.
-
- BRC No: 54461
- -------------
- When a view is removed from a still-open hierarchy, the window's
- fTarget and gTarget are not checked for validity.
-
- BRC No: 54462
- -------------
- There is insufficient support for adding/removing views in already-
- open view hierarchies.
-
- BRC No: 54463
- -------------
- TScroller should override ForceRedraw to also force its scrollbars
- to redraw. They are created behind your back, so should be
- considered part of the scroller.
-
- BRC No: 54464
- -------------
- TView.BeInScroller should call itsScroller.SetScrollLimits with
- FALSE for the "drawScrollBars" parameter
-
- BRC No: 54465
- -------------
- MABuild should have an option to link with CRunTime.o rather than
- RunTime.o to support those who have C or C++ code, but a Pascal
- main program.
-
- BRC No: 54466
- -------------
- MABuild should pass the compile-time variables TRUE=1 and
- FALSE=0 to C and C++. Types.h only defines true and false
- (lowercase).
-
- BRC No: 54467
- -------------
- SectVRect returns true when the rectangles are just touching
- (bottom of one = top of other), unlike B5 and the toolbox' SectRect.
-
-